-
Notifications
You must be signed in to change notification settings - Fork 745
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[Traefik] Add traefik check #64
Conversation
795c2ca
to
30da1b0
Compare
hey @hkaj, one of the Traefik maintainers here. :) First of all: Even though I don't use DataDog, it's great to see you're providing better integration with Traefik. Appreciated! Skimming over the PR, I saw that you apparently maintain a mapping of Prometheus metrics used in Traefik. Out of curiosity, is there any plan on how to stay in sync with Traefik's set of metrics from your end? FWIW, there's PR traefik/traefik#1504 in-flight right now to add metrics for the retry counter, and we are probably going to have a few more over time. Thanks again! |
Hey @timoreimann New metrics will need to be added manually to the check, and these updates will be provided on a best-effort basis. The community can handle it, or create issues to notify us/other users about new metrics to add. This is already the process for kube-state-metrics for example. We don't want to collect all metrics by default because the prometheus format is quite different from the datadog one, so we need to be able to transform metrics to something that better fits our model. And we can't do it retroactively because it would change metrics that people could already be using for alerting (and thus triggering un-wanted alerts). Does that make sense? |
@hkaj makes absolute sense. 👍 Thanks for the explanation. |
30da1b0
to
4070caa
Compare
that took me long enough to finish it... Can I get a review on this one @truthbk ? |
This issue has been automatically marked as stale because it has not had activity in the last 60 days. It will be closed if no further activity occurs in the next two weeks. Thank you for participating in the Datadog open source community. If you need more time, or have any questions, let us know. |
Spoke with Haissam. He still plans to have this merged, but needs a bit of additional time. Removing the stale tag. |
I'd love to see status count( how many 20x, 30x, 40x, 50x) added to this. |
This issue has been automatically marked as stale because it has not had activity in the last 60 days. It will be closed if no further activity occurs in the next two weeks. Thank you for participating in the Datadog open source community. If you need more time, or have any questions, let us know. |
@jeremy-lq any update ? :) |
This issue has been automatically marked as stale because it has not had activity in the last 60 days. It will be closed if no further activity occurs in the next two weeks. Thank you for participating in the Datadog open source community. If you need more time, or have any questions, let us know. |
What does this PR do?
Adds a Traefik check.
Motivation
prometheus_check
needs a few checks using it so we can see what's to improve